Skip to content

MCUEVSE-120 add defined parameter serv values for low current time an…#141

Open
eternalamenity wants to merge 2 commits intomasterfrom
feature/MCUEVSE-120-add-defined-low-current-time-and-threshold-for-charger
Open

MCUEVSE-120 add defined parameter serv values for low current time an…#141
eternalamenity wants to merge 2 commits intomasterfrom
feature/MCUEVSE-120-add-defined-low-current-time-and-threshold-for-charger

Conversation

@eternalamenity
Copy link
Contributor

…d threshold - charger

Copy link
Contributor

@boguslaw-wojcik boguslaw-wojcik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See comment.

| `used_sensors` | Used sensors | `thermostat` | `string` | `select` | `room_sensor`, `floor_sensor`, `floor_and_room_sensor` | Select which sensors should be used by the thermostat to calculate heating hysteresis. |
| `floor_sensor_resistance` | Floor sensor resistance | `thermostat` | `int` | `select` | | Set the floor sensor resistance in `kΩ`, if it is installed. |
| `low_current_time` | Low current time | `chargepoint` | `int` | `input` | | Set the time after which the low current detection feature will stop an ongoing transaction in minutes. |
| `low_current_threshold` | Low current threshold | `chargepoint` | `int` | `input` | | Stops the transaction if for a prolonged time the EV is charging with a low current. |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe: ...with a current lower than configured number of Ampers... or similar. There is no information we set the threshold to ampers. Please also check which one when set to 0 will disable the feature altogether.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

BudzikM
BudzikM previously approved these changes Jan 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants